feat: add battery check before system updates#142
Conversation
1. Added battery status check in UpdateWorker before starting upgrades 2. Show notification if battery is not plugged in 3. Added new signal startDoUpgrade to properly handle dialog closing 4. Updated translations for new battery check message 5. Modified update dialog to close only after upgrade starts This change prevents system updates when running on battery to avoid potential issues from power loss during update process. The dialog now closes at the right time after the upgrade begins rather than immediately when clicking the button. feat: 在系统更新前添加电池检查 1. 在UpdateWorker中添加升级前的电池状态检查 2. 如果未插入电源则显示通知 3. 添加新的startDoUpgrade信号以正确处理对话框关闭 4. 更新电池检查消息的翻译 5. 修改更新对话框只在升级开始后关闭 此更改可防止在电池供电时进行系统更新,以避免更新过程中断电可能导致的问 题。对话框现在会在升级开始后而不是点击按钮时立即关闭。 pms: Bug-320697
deepin pr auto review代码审查意见:
以上是本次代码审查的主要意见,希望能够对您的开发工作有所帮助。 |
|
@xionglinlin: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 18202781743, xionglinlin The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
This change prevents system updates when running on battery to avoid potential issues from power loss during update process. The dialog now closes at the right time after the upgrade begins rather than immediately when clicking the button.
feat: 在系统更新前添加电池检查
此更改可防止在电池供电时进行系统更新,以避免更新过程中断电可能导致的问
题。对话框现在会在升级开始后而不是点击按钮时立即关闭。
pms: Bug-320697